Skip to content

Conversation

@soustruh
Copy link
Contributor

Summary

Simplifies CI/CD workflows from 4 files to 2 by consolidating common logic.

Changes

  • Unified deploy workflow: Single deploy.yml with manual dropdown (test/production) replaces separate test/prod workflows
  • Simplified CI: push.yml runs on all branches except main (follows cookiecutter pattern)
  • Docs on release only: Documentation generated only during production deploys (not on every main push)

Files Changed

  • Added: .github/workflows/push.yml (renamed from push_dev.yml, skips main branch)
  • Modified: .github/workflows/deploy.yml (unified test/prod with dropdown + conditional docs)
  • Deleted: deploy_to_test.yml, push_main.yml (merged into above)

Testing

Manual workflow dispatch now available with environment dropdown:

  • Select test → Deploy to Test PyPI (no docs)
  • Select production → Deploy to Production PyPI + generate docs
  • GitHub Releases still auto-trigger production deploys
    Net result: -60 lines YAML, same functionality, cleaner structure.

@soustruh soustruh requested a review from a team January 21, 2026 12:36
Copy link

@matyas-jirat-keboola matyas-jirat-keboola left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@soustruh soustruh merged commit 105ba7b into main Jan 21, 2026
2 checks passed
@soustruh soustruh deleted the uv branch January 21, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants